Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feat/218] 매칭 우선순위 세부 작업 진행 #221

Merged
merged 3 commits into from
Sep 20, 2024
Merged

Conversation

rimi3226
Copy link
Contributor

@rimi3226 rimi3226 commented Sep 19, 2024

🚀 개요

게임모드에 따른 매칭 우선순위 세부 작업 진행

🔍 변경사항

  • 칼바람 우선순위 계산 변경
  • 자유랭크 우선순위 계산 변경
  • 개인랭크 우선순위 계산 변경

⏳ 작업 내용

  • 칼바람 : 포지션 선택 제외하기 -> 포지션 점수 기본값으로 주고 시작하기
  • 자유랭크 : 마스터 이상 플레이어는 골드 이하의 플레이어와 팀 구성할 수 없음
  • 자유랭크 : 골드 이하의 플레이어는 마스터 이상의 플레이어와 게임 불가능
  • 일반랭크 : 다이아 이하 구간이 마스터 이상 구간과 잡히지 않도록 예외처리
  • 일반랭크 : 마스터 이상 구간 -> 개인 랭크 못함
  • 주/부/ 원하는 포지션 값과 gameStyleIdList 값이 null이라도 에러 발생하지 않도록 처리

📝 논의사항

  1. 랭크게임 조건 추가 + 레벨 30 이상, 보유 챔피언 20개 이상 참여 가능 확인하기
    -> 불가능. 보유 챔피언이 20개 이상이라는 정보를 받아오는 API가 없음. 프론트측에서 팝업으로 알려주는 수밖에 없음
  2. 일반랭크 : 다이아 이하 구간이 마스터 이상 구간과 잡히지 않도록 예외처리를 마스터 이상 구간이 있을 경우 무조건 우선순위 값을 0으로 하도록 처리함. 하지만 이 부분은 프론트에서 미리 예외처리 해서 API 요청하도록 수정해야함.

@rimi3226 rimi3226 self-assigned this Sep 19, 2024
@rimi3226 rimi3226 linked an issue Sep 19, 2024 that may be closed by this pull request
5 tasks
@Eunjin3395
Copy link
Member

mainP, subP, wantP가 이제 nullable이 되었는데 그럼 빠른대전, 솔로랭크, 자유 랭크 모드에서도 포지션이 null일 수 있는건가요?! 만약 null이 가능하다면 우선순위 점수는 어떻게 계산되는건가요?

@rimi3226
Copy link
Contributor Author

mainP, subP, wantP가 이제 nullable이 되었는데 그럼 빠른대전, 솔로랭크, 자유 랭크 모드에서도 포지션이 null일 수 있는건가요?! 만약 null이 가능하다면 우선순위 점수는 어떻게 계산되는건가요?

수정했습니다!

@rimi3226 rimi3226 merged commit d331242 into develop Sep 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

✨ [Feat] 매칭 우선순위 세부 작업 진행
2 participants